Skip to content

Correct context for CancellableSOCache listener#83021

Merged
DaveCTurner merged 4 commits intoelastic:masterfrom
DaveCTurner:2022-01-25-CancellableSingleObjectCache-thread-context
Jan 26, 2022
Merged

Correct context for CancellableSOCache listener#83021
DaveCTurner merged 4 commits intoelastic:masterfrom
DaveCTurner:2022-01-25-CancellableSingleObjectCache-thread-context

Conversation

@DaveCTurner
Copy link
Contributor

Today the CancellableSingleObjectCache completes its listeners in the
thread context of the get() call that actually computes the value
which will be the correct context only if no batching took place. With
this commit we make sure to complete each listener in the context in
which it was passed to the corresponding get() call.

Today the `CancellableSingleObjectCache` completes its listeners in the
thread context of the `get()` call that actually computes the value
which will be the correct context only if no batching took place. With
this commit we make sure to complete each listener in the context in
which it was passed to the corresponding `get()` call.
@DaveCTurner DaveCTurner added >bug :Core/Infra/Stats Statistics tracking and retrieval APIs v8.0.0 v8.1.0 v7.17.1 labels Jan 25, 2022
@elasticmachine elasticmachine added the Team:Data Management DO NOT USE. Meta label for data/management team. This team no longer exists. label Jan 25, 2022
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-data-management (Team:Data Management)

@elasticsearchmachine
Copy link
Collaborator

Hi @DaveCTurner, I've created a changelog YAML for you.

@DaveCTurner DaveCTurner merged commit 5c684c3 into elastic:master Jan 26, 2022
@DaveCTurner DaveCTurner deleted the 2022-01-25-CancellableSingleObjectCache-thread-context branch January 26, 2022 16:40
@elasticsearchmachine
Copy link
Collaborator

💔 Backport failed

Status Branch Result
8.0 Commit could not be cherrypicked due to conflicts
7.17 Commit could not be cherrypicked due to conflicts

You can use sqren/backport to manually backport by running backport --upstream elastic/elasticsearch --pr 83021

DaveCTurner added a commit to DaveCTurner/elasticsearch that referenced this pull request Jan 26, 2022
Today the `CancellableSingleObjectCache` completes its listeners in the
thread context of the `get()` call that actually computes the value
which will be the correct context only if no batching took place. With
this commit we make sure to complete each listener in the context in
which it was passed to the corresponding `get()` call.
DaveCTurner added a commit to DaveCTurner/elasticsearch that referenced this pull request Jan 26, 2022
Today the `CancellableSingleObjectCache` completes its listeners in the
thread context of the `get()` call that actually computes the value
which will be the correct context only if no batching took place. With
this commit we make sure to complete each listener in the context in
which it was passed to the corresponding `get()` call.
elasticsearchmachine pushed a commit that referenced this pull request Jan 26, 2022
Today the `CancellableSingleObjectCache` completes its listeners in the
thread context of the `get()` call that actually computes the value
which will be the correct context only if no batching took place. With
this commit we make sure to complete each listener in the context in
which it was passed to the corresponding `get()` call.
elasticsearchmachine pushed a commit that referenced this pull request Jan 26, 2022
* Correct context for CancellableSOCache listener (#83021)

Today the `CancellableSingleObjectCache` completes its listeners in the
thread context of the `get()` call that actually computes the value
which will be the correct context only if no batching took place. With
this commit we make sure to complete each listener in the context in
which it was passed to the corresponding `get()` call.

* Fix changelog YAML

* Spotless
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

>bug :Core/Infra/Stats Statistics tracking and retrieval APIs Team:Data Management DO NOT USE. Meta label for data/management team. This team no longer exists. v7.17.0 v8.0.0 v8.1.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants